Skip to content

Add e2e coverage for cquery streamed_proto path (#219)#333

Merged
tinder-maxwellelliott merged 1 commit intomasterfrom
cquery-streamed-proto-test-219
Apr 27, 2026
Merged

Add e2e coverage for cquery streamed_proto path (#219)#333
tinder-maxwellelliott merged 1 commit intomasterfrom
cquery-streamed-proto-test-219

Conversation

@tinder-maxwellelliott
Copy link
Copy Markdown
Collaborator

Summary

  • Adds testGenerateHashesWithCqueryStreamedProto to E2ETest.kt, an explicit end-to-end test that exercises generate-hashes --useCquery against the lightweight distance_metrics workspace. This guards the AnalysisProtosV2.CqueryResult.parseDelimitedFrom streaming loop in BazelQueryService.query() (Bazel 7.0.0+).
  • The underlying fix for #219 (streamed_proto support for cquery) already landed in Use --output=streamed_proto and --output_file if supported #278; existing cquery e2e tests cover bzlmod / multi-platform fixtures but none isolate the simple --useCquery codepath.
  • Bumps version 19.0.0 → 19.0.1 in MODULE.bazel and README.md.

Test plan

  • bazel test //cli:E2ETest --test_filter='com.bazel_diff.e2e.E2ETest#testGenerateHashesWithCqueryStreamedProto' passes locally on Bazel 8.5.1
  • CI green

🤖 Generated with Claude Code

….0.1

The fix for #219 (streamed_proto support for cquery) landed in #278;
this adds an explicit e2e test that exercises the --useCquery code path
to guard against regressions in the parseDelimitedFrom loop.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tinder-maxwellelliott tinder-maxwellelliott merged commit 4a49bbe into master Apr 27, 2026
24 of 29 checks passed
@tinder-maxwellelliott tinder-maxwellelliott deleted the cquery-streamed-proto-test-219 branch April 27, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant